Spectrogram example notebook #181
Conversation
SundarRajan28
commented
Jul 12, 2024
SundarRajan28
commented
- Adding an example notebook for audio processing with spectrogram augmentation
SundarRajan28
requested review from
kiritigowda,
rrawther and
a team
as code owners
There was a problem hiding this comment.
@SundarRajan28 I am seeing a failure in the build stage for this notebook example:
Verify graph failed
[ERR] {rocalVerify} Invalid layout
---------------------------------------------------------------------------
NameError Traceback (most recent call last)
Cell In[13], [line 1](vscode-notebook-cell:?execution_count=13&line=1)
----> [1](vscode-notebook-cell:?execution_count=13&line=1) audio_pipeline.build()
[2](vscode-notebook-cell:?execution_count=13&line=2) audioIteratorPipeline = ROCALAudioIterator(audio_pipeline)
File /usr/local/lib/python3.10/dist-packages/amd/rocal/pipeline.py:133, in Pipeline.build(self)
[131](https://vscode-remote+ssh-002dremote-002bkapu.vscode-resource.vscode-cdn.net/usr/local/lib/python3.10/dist-packages/amd/rocal/pipeline.py:131) if (status != types.OK):
[132](https://vscode-remote+ssh-002dremote-002bkapu.vscode-resource.vscode-cdn.net/usr/local/lib/python3.10/dist-packages/amd/rocal/pipeline.py:132) print("Verify graph failed")
--> [133](https://vscode-remote+ssh-002dremote-002bkapu.vscode-resource.vscode-cdn.net/usr/local/lib/python3.10/dist-packages/amd/rocal/pipeline.py:133) exit(0)
[134](https://vscode-remote+ssh-002dremote-002bkapu.vscode-resource.vscode-cdn.net/usr/local/lib/python3.10/dist-packages/amd/rocal/pipeline.py:134) return self
NameError: name 'exit' is not defined
|
@SundarRajan28 also please add a readme to state pre reqs like librosa, matplotlib modules etc. |
|
@LakshmiKumar23 Can you share how you are building the example notebook so that I can replicate the issue? I have not encountered this error. |
@SundarRajan28 I am running it on my vscode |
There was a problem hiding this comment.
@SundarRajan28 : Please address the review comments so we can merge this PR
|
@rrawther I have resolved the review comments |
|
@LakshmiKumar23: Can you please see if review comments are addressed and approve if it is ready to be merged |
There was a problem hiding this comment.
@rrawther @kiritigowda this notebook has been tested with package install for rpp and mivisionx . Can be merged
